Skip to content

Fix chat panel horizontal scroll + padding#31

Merged
quan0715 merged 1 commit intomainfrom
codex/fix-chat-panel-scroll-padding
Feb 8, 2026
Merged

Fix chat panel horizontal scroll + padding#31
quan0715 merged 1 commit intomainfrom
codex/fix-chat-panel-scroll-padding

Conversation

@quan0715
Copy link
Copy Markdown
Owner

@quan0715 quan0715 commented Feb 8, 2026

  • Enable horizontal scrollbar in ScrollArea (opt-in via prop)\n- Use horizontal+vertical scrollbars in chat message panel\n- Add left/right padding for chat messages

Copilot AI review requested due to automatic review settings February 8, 2026 09:21
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the frontend scrolling utility and chat UI to better handle wide content (e.g., long code lines) by allowing opt-in horizontal scrollbars and adding more consistent horizontal padding in the chat message list.

Changes:

  • Added an opt-in scrollbar mode to the shared ScrollArea component (vertical | horizontal | both).
  • Enabled both horizontal and vertical scrollbars in the chat messages panel.
  • Adjusted chat message container padding from p-3 to px-4 py-3.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
frontend/src/components/ui/scroll-area.tsx Adds configurable scrollbar rendering (vertical/horizontal/both) while keeping default behavior vertical-only.
frontend/src/components/chat/ChatPanel.tsx Enables both scrollbars for the message list and tweaks padding to improve layout/spacing.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@quan0715 quan0715 merged commit c727abc into main Feb 8, 2026
8 checks passed
@quan0715 quan0715 deleted the codex/fix-chat-panel-scroll-padding branch February 8, 2026 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants